Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: ugly hack to debug Windows builds in CI fail #2468

Closed
wants to merge 1 commit into from

Conversation

neteler
Copy link
Member

@neteler neteler commented Jun 28, 2022

Attempt to see more (error) messages.

Related to bug #2454

Attempt to see more (error) messages.

Related to bug OSGeo#2454
@neteler neteler added the CI Continuous integration label Jun 28, 2022
@neteler neteler self-assigned this Jun 28, 2022
@neteler
Copy link
Member Author

neteler commented Jun 28, 2022

It is weird...:

2022-06-28T08:25:24.7528731Z C:/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: warning: --export-dynamic is not supported for PE+ targets, did you mean --export-all-symbols? 2022-06-28T08:25:24.8306939Z if [ "/d/a/grass/grass/dist.x86_64-w64-mingw32/bin/d.labels.exe" != "" ] ; then GISRC=/d/a/grass/grass/dist.x86_64-w64-mingw32/demolocation/.grassrc83 GISBASE=D:/a/grass/grass/dist.x86_64-w64-mingw32 PATH="/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/scripts:$PATH" PYTHONPATH="D:/a/grass/grass/dist.x86_64-w64-mingw32/etc/python;D:/a/grass/grass/dist.x86_64-w64-mingw32/gui/wxpython;$PYTHONPATH" PATH="/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/scripts:/d/a/grass/grass/dist.x86_64-w64-mingw32/lib:/d/a/grass/grass/dist.x86_64-w64-mingw32/lib:/c/OSGeo4W/bin:/usr/bin:/mingw64/bin" LC_ALL=C LANG=C LANGUAGE=C /d/a/grass/grass/dist.x86_64-w64-mingw32/bin/d.labels.exe --html-description | grep -v '</body>\|</html>' ; fi 2022-06-28T08:25:24.8990288Z make[3]: *** [../../include/Make/Html.make:14: d.labels.tmp.html] Error 1

@neteler neteler added the windows Microsoft Windows specific label Jun 28, 2022
@hellik
Copy link
Member

hellik commented Jun 28, 2022

there are 2 issues in the configure

checking whether to use Cairo... "yes"
Package cairo was not found in the pkg-config search path.
Perhaps you should add the directory containing `cairo.pc'
to the PKG_CONFIG_PATH environment variable
Package 'cairo', required by 'virtual:world', not found
checking for location of cairo includes... /d/a/grass/grass/include
checking cairo.h usability... yes
checking cairo.h presence... yes
checking for cairo.h... yes
Package cairo was not found in the pkg-config search path.
Perhaps you should add the directory containing `cairo.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of cairo library... 
checking for cairo linking flags... -L/d/a/grass/grass/mswindows/osgeo4w/lib -lcairo -lfontconfig
Package 'cairo', required by 'virtual:world', not found
checking for cairo_create... yes
checking for cairo_xlib_surface_create_with_xrender_format... no
checking for cairo_xlib_surface_get_xrender_format... no
checking whether to use FreeType... "yes"
Package freetype2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `freetype2.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of FreeType includes... /mingw64/include/freetype2
Package 'freetype2', required by 'virtual:world', not found
checking ft2build.h usability... yes
checking ft2build.h presence... yes
checking for ft2build.h... yes
Package freetype2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `freetype2.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of FreeType library... 
Package 'freetype2', required by 'virtual:world', not found
checking for FT_Init_FreeType in -lfreetype... yes

not sure what virtual:world is for.

though, config seems to be ok

  Cairo support:              yes
  DWG support:                no
  FFTW support:               yes
  FreeType support:           yes

@hellik
Copy link
Member

hellik commented Jun 28, 2022

it seems to be an issue of the build environment (results of googling the error message)

@hellik
Copy link
Member

hellik commented Jun 28, 2022

looking here in my build environment

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config freetype2 --libs
-LC:/msys64/mingw64/lib -lfreetype

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config freetype2 --cflags
-IC:/msys64/mingw64/include/freetype2 -IC:/msys64/mingw64/include -IC:/msys64/mingw64/include/libpng16 -IC:/msys64/mingw64/include/harfbuzz -IC:/msys64/mingw64/include/glib-2.0 -IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config cairo --libs
-LC:/msys64/mingw64/lib -lcairo

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config cairo --cflags
-IC:/msys64/mingw64/include/cairo -IC:/msys64/mingw64/include/lzo -IC:/msys64/mingw64/include -IC:/msys64/mingw64/include/libpng16 -IC:/msys64/mingw64/include/freetype2 -IC:/msys64/mingw64/include/harfbuzz -IC:/msys64/mingw64/include/glib-2.0 -IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields -IC:/msys64/mingw64/include/pixman-1

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ echo $PKG_CONFIG_PATH
/mingw64/lib/pkgconfig:/mingw64/share/pkgconfig

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$

no chance to do a local test build here.

not sure how to test it in the CI build environment

@ninsbl any idea?

@hellik
Copy link
Member

hellik commented Jun 28, 2022

maybe some hints in a github issue

fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

@neteler
Copy link
Member Author

neteler commented Jun 28, 2022

Here is the full log (hopefully accessible):
https://github.com/OSGeo/grass/commit/6866c3fb6b065796ef7b5de71b5d8389c55db735/checks/7088374248/logs

Here also as a ZIP file: ci_windows_mingw_errors.zip

@neteler
Copy link
Member Author

neteler commented Jun 29, 2022

maybe some hints in a github issue

fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

As far as I see this is already used - see:

run: C:\msys64\usr\bin\pacman.exe --noconfirm -S tar libintl make bison flex diffutils git dos2unix zip mingw-w64-x86_64-toolchain mingw-w64-x86_64-cairo mingw-w64-x86_64-fftw mingw-w64-x86_64-lapack mingw-w64-x86_64-pkg-config mingw-w64-x86_64-gcc mingw-w64-x86_64-ccache mingw-w64-x86_64-zlib mingw-w64-x86_64-libiconv mingw-w64-x86_64-bzip2 mingw-w64-x86_64-gettext mingw-w64-x86_64-libsystre mingw-w64-x86_64-libtre-git mingw-w64-x86_64-libwinpthread-git mingw-w64-x86_64-libpng mingw-w64-x86_64-pcre mingw-w64-x86_64-python3-six

@hellik
Copy link
Member

hellik commented Jun 29, 2022

maybe some hints in a github issue
fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

As far as I see this is already used - see:

run: C:\msys64\usr\bin\pacman.exe --noconfirm -S tar libintl make bison flex diffutils git dos2unix zip mingw-w64-x86_64-toolchain mingw-w64-x86_64-cairo mingw-w64-x86_64-fftw mingw-w64-x86_64-lapack mingw-w64-x86_64-pkg-config mingw-w64-x86_64-gcc mingw-w64-x86_64-ccache mingw-w64-x86_64-zlib mingw-w64-x86_64-libiconv mingw-w64-x86_64-bzip2 mingw-w64-x86_64-gettext mingw-w64-x86_64-libsystre mingw-w64-x86_64-libtre-git mingw-w64-x86_64-libwinpthread-git mingw-w64-x86_64-libpng mingw-w64-x86_64-pcre mingw-w64-x86_64-python3-six

Though not sure mingw-w64-x86_64 is used for compiling

@neteler
Copy link
Member Author

neteler commented Jun 30, 2022

Though not sure mingw-w64-x86_64 is used for compiling

@hellik your remark isn't clear to me...

@ninsbl
Copy link
Member

ninsbl commented Aug 1, 2022

Sorry for my silence n this. For some reasons I do not receive GH notifications any more although I am watching issues...

Probably the issue is really the PKG_CONFIG_PATH., see e.g. msys2/setup-msys2#171

Will try to have a look at this the comming days...

@ninsbl
Copy link
Member

ninsbl commented Aug 2, 2022

The culprit is likely here:
https://github.com/OSGeo/grass/runs/7513175054?check_suite_focus=true#step:5:242

Using msys2-setup@v2 action seems to help (tests are currently running in my fork...)...

I suggest to abandon this PR in favor of #2508...

@neteler
Copy link
Member Author

neteler commented Aug 3, 2022

I suggest to abandon this PR in favor of #2508...

Yes, it was just a test.
Closing.

@neteler neteler closed this Aug 3, 2022
@neteler neteler deleted the ci_windows_debug_only branch August 3, 2022 14:40
@neteler neteler added the invalid This doesn't seem to be a bug label Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration invalid This doesn't seem to be a bug windows Microsoft Windows specific
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants